Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

ci: add job summary to check status #211

Merged
merged 10 commits into from
May 15, 2024
Merged

ci: add job summary to check status #211

merged 10 commits into from
May 15, 2024

Conversation

rdhar
Copy link
Member

@rdhar rdhar commented May 15, 2024

Signed-off-by: Rishav Dhar 19497993+rdhar@users.noreply.github.com

What kind of change does this PR introduce?

What is the current behaviour?

What is the new behaviour?

Does this PR introduce a breaking change?

@rdhar
add initial draft with limited tests
1d072a9 
Signed-off-by: Rishav Dhar <19497993+rdhar@users.noreply.github.com>
@github-actions github-actions bot added the tf:plan Pull requests that plan TF code label May 15, 2024
rdhar added 9 commits May 15, 2024 10:33
@rdhar
replace statuses with checks permission
a84f478 
Signed-off-by: Rishav Dhar <19497993+rdhar@users.noreply.github.com>
@rdhar
retry with check_id instead of run_id
33a2a28 
Signed-off-by: Rishav Dhar <19497993+rdhar@users.noreply.github.com>
@rdhar
merge script into "comment_tf_output" due to overlap of concerns
5b7adb2 
Signed-off-by: Rishav Dhar <19497993+rdhar@users.noreply.github.com>
@rdhar
re-introduce placeholder summary as required
ef96c74 
Signed-off-by: Rishav Dhar <19497993+rdhar@users.noreply.github.com>
@rdhar
test with reduced checks: read permission
ce0b45d 
Signed-off-by: Rishav Dhar <19497993+rdhar@users.noreply.github.com>
@rdhar
add checks: write permission to all example workflows
2972d2b 
Signed-off-by: Rishav Dhar <19497993+rdhar@users.noreply.github.com>
@rdhar
display comment output as a job summary
5859f87 
Signed-off-by: Rishav Dhar <19497993+rdhar@users.noreply.github.com>
@rdhar
fix reference to whole comment_body instead of partial comment_output
7b25873 
Signed-off-by: Rishav Dhar <19497993+rdhar@users.noreply.github.com>
@rdhar
revert tests ready for merge
41cea38 
Signed-off-by: Rishav Dhar <19497993+rdhar@users.noreply.github.com>
@github-actions GitHub Actions
Copy link

{"tf":"plan","chdir":"10_fail_invalid_resource_type"}

Error: Invalid resource type
TF Tests by @rdhar via pull_request at 2024-05-15T10:28:07Z.
 
Error: Invalid resource type

  on main.tf line 1, in resource "random_pets" "name":
   1: resource "random_pets" "name" {

The provider hashicorp/random does not support resource type "random_pets".
Did you mean "random_pet"?

@github-actions GitHub Actions
Copy link

{"tf":"plan","chdir":"11_fail_data_source_error"}

Error: Read local file data source error
TF Tests by @rdhar via pull_request at 2024-05-15T10:28:07Z.
 
Error: Read local file data source error

  with data.local_file.path,
  on main.tf line 1, in data "local_file" "path":
   1: data "local_file" "path" {

The file at given path cannot be read.

+Original Error: open /error: no such file or directory

@github-actions GitHub Actions
Copy link

{"tf":"plan","chdir":"10_fail_invalid_resource_type"}

Error: Invalid resource type
TF Tests by @rdhar via pull_request at 2024-05-15T10:28:07Z.
 
Error: Invalid resource type

  on main.tf line 1, in resource "random_pets" "name":
   1: resource "random_pets" "name" {

The provider hashicorp/random does not support resource type "random_pets".
Did you mean "random_pet"?

@github-actions GitHub Actions
Copy link

{"tf":"plan","chdir":"11_fail_data_source_error"}

Error: Read local file data source error
TF Tests by @rdhar via pull_request at 2024-05-15T10:28:07Z.
 
Error: Read local file data source error

  with data.local_file.path,
  on main.tf line 1, in data "local_file" "path":
   1: data "local_file" "path" {

The file at given path cannot be read.

+Original Error: open /error: no such file or directory

@rdhar rdhar merged commit 6306480 into main May 15, 2024
12 checks passed
@rdhar rdhar deleted the ci--add-job-summary-to-check-status branch May 15, 2024 10:30
@github-actions github-actions bot added the tf:apply Pull requests that apply TF code label May 15, 2024
@github-actions GitHub Actions
Copy link

{"tf":"apply","chdir":"52_pass_format_diff"}

Format check diff. 
main.tf
--- old/main.tf
+++ new/main.tf
@@ -1,3 +1,3 @@
 resource "random_pet" "name" {
-  count =1
+  count = 1
 }
Apply complete! Resources: 1 added, 0 changed, 0 destroyed.
TF Tests by @rdhar via pull_request at 2024-05-15T10:30:20Z.
 
Apply complete! Resources: 1 added, 0 changed, 0 destroyed.

@github-actions GitHub Actions
Copy link

{"tf":"apply","chdir":"52_pass_format_diff"}

Format check diff. 
main.tf
--- old/main.tf
+++ new/main.tf
@@ -1,3 +1,3 @@
 resource "random_pet" "name" {
-  count =1
+  count = 1
 }
Apply complete! Resources: 1 added, 0 changed, 0 destroyed.
TF Tests by @rdhar via pull_request at 2024-05-15T10:30:20Z.
 
Apply complete! Resources: 1 added, 0 changed, 0 destroyed.

@github-actions GitHub Actions
Copy link

{"tf":"apply","chdir":"50_pass_one"}

Apply complete! Resources: 1 added, 0 changed, 0 destroyed.
TF Tests by @rdhar via pull_request at 2024-05-15T10:30:20Z.
 
Apply complete! Resources: 1 added, 0 changed, 0 destroyed.

@github-actions GitHub Actions
Copy link

{"tf":"apply","chdir":"50_pass_one"}

Apply complete! Resources: 1 added, 0 changed, 0 destroyed.
TF Tests by @rdhar via pull_request at 2024-05-15T10:30:20Z.
 
Apply complete! Resources: 1 added, 0 changed, 0 destroyed.

@github-actions GitHub Actions
Copy link

{"tf":"apply","chdir":"51_pass_character_limit"}

Apply complete! Resources: 10000 added, 0 changed, 0 destroyed.
TF Tests by @rdhar via pull_request at 2024-05-15T10:30:20Z.
 
Apply complete! Resources: 10000 added, 0 changed, 0 destroyed.

@github-actions GitHub Actions
Copy link

{"tf":"apply","chdir":"51_pass_character_limit"}

Apply complete! Resources: 10000 added, 0 changed, 0 destroyed.
TF Tests by @rdhar via pull_request at 2024-05-15T10:30:20Z.
 
Apply complete! Resources: 10000 added, 0 changed, 0 destroyed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
tf:apply Pull requests that apply TF code tf:plan Pull requests that plan TF code
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@rdhar